Skip to content

fix(skills): derive bug-triage target repo from the checkout - #5

Merged
bfoxhound merged 1 commit into
mainfrom
claude/pnpm-lockfile-migration-5c6q07
Jul 28, 2026
Merged

fix(skills): derive bug-triage target repo from the checkout#5
bfoxhound merged 1 commit into
mainfrom
claude/pnpm-lockfile-migration-5c6q07

Conversation

@bfoxhound

Copy link
Copy Markdown
Owner

What

skills/bug-triage/SKILL.md hard-coded --repo aoagents/ReverbCode in 21 places. This checkout is bfoxhound/agent-orchestrator, so anyone running the skill here had every gh call silently target a different repository: duplicate search queried the wrong issue tracker (and returned nothing), new issues were filed elsewhere, and the screenshot raw-URL template pointed at asset branches pushed to another repo.

One issue, one PR, per AGENTS.md.

The fix

Resolve the slug once in pre-flight and use $REPO everywhere:

REPO="${AO_TRIAGE_REPO:-$(gh repo view --json nameWithOwner -q .nameWithOwner)}"
  • Default — derived from the checkout, so triage operates on the tree being triaged. Correct regardless of which fork or rename you are standing in.
  • AO_TRIAGE_REPO override — preserves the old always-file-upstream intent as an explicit choice (AO_TRIAGE_REPO=aoagents/ReverbCode), and covers the Appendix B clone-less flow where gh repo view has no checkout to read.

All 21 sites were converted and reviewed individually rather than by blanket substitution. The ones needing care:

  • --repo flags adjacent to <<'EOF' heredocs (§4, §5c) sit outside the quoted heredoc, so "$REPO" expands normally.
  • The ao spawn --prompt "…" worker instruction (§5f) is a double-quoted string, so $REPO resolves at spawn time and the worker receives the concrete slug.
  • The two body-template URLs — the screenshot raw link (§5b) and the issue-ref linkify rule (Formatting Rules) — move to $REPO too; a link template aimed at a fixed repo misfiles the same way a gh flag does.
  • gh api paths are now consistently double-quoted ("repos/$REPO/git/trees/main?recursive=1"), which also protects the ? from glob-happy shells.

Product-identity prose is deliberately untouched: ReverbCode as the product name, port 3001, the Go daemon paths, and the Electron frontend all match this tree (verified against backend/internal/config/config.go, backend/internal/session_manager/manager.go, frontend/forge.config.ts, and DESIGN.md). Only the repo slug was wrong.

Verification

  • grep -c 'aoagents/ReverbCode' → exactly 1 remaining literal, the intentional AO_TRIAGE_REPO example in the pre-flight bullet.
  • Every $REPO construction passes bash -n.
  • Prettier clean.

Docs-only change; no code paths affected.


Generated by Claude Code

bug-triage hard-coded aoagents/ReverbCode in 21 places. Run from any
other checkout — this repo is bfoxhound/agent-orchestrator — every gh
call silently targeted the wrong repository: duplicate search returned
nothing from the wrong issue tracker, new issues landed on another repo,
and screenshot raw URLs pointed at asset branches that were pushed
elsewhere.

Resolve the slug once in pre-flight and use it everywhere:

  REPO="${AO_TRIAGE_REPO:-$(gh repo view --json nameWithOwner -q .nameWithOwner)}"

Deriving from the checkout keeps triage on the tree being triaged. The
old always-file-upstream behaviour stays available as an explicit
choice via AO_TRIAGE_REPO, which also covers the clone-less Appendix B
flow where gh repo view has nothing to read.

The two body-template URLs (screenshot raw link, issue-ref linkify rule)
move to $REPO as well, since a link template pointing at a fixed repo
has the same misfiling problem as a gh flag. Product-identity prose
(ReverbCode, port 3001, Go paths) is untouched — it matches this tree
and was never the bug.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VjPWzWR3yvT35opqeixiWy
@github-actions

Copy link
Copy Markdown

Test Coverage Report

No TypeScript source files changed in this PR.

@bfoxhound
bfoxhound marked this pull request as ready for review July 28, 2026 03:23
@bfoxhound
bfoxhound merged commit 8cac622 into main Jul 28, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants